Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relax backend for default-constructed event #582

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Jul 12, 2024

Relax the requirements on the backend for a default-constructed event object. Previously, we required the backend for such an event to be the same as the backend for the default device. After some implementation experience, we decided that this is not practical. Requiring interoperation with a particular backend constrains the implementation too much. It also seems arbitrary to require the default-constructed event to have a particular backend. To address these concerns, we now say that the backend that is associated with a default-constructed event is implementation defined.

Relax the requirements on the backend for a default-constructed `event`
object.  Previously, we required the backend for such an event to be the
same as the backend for the default device.  After some implementation
experience, we decided that this is not practical.  Requiring
interoperation with a particular backend constrains the implementation
too much.  It also seems arbitrary to require the default-constructed
event to have a particular backend.  To address these concerns, we now
say that the backend that is associated with a default-constructed
event is implementation defined.
Copy link
Contributor

@illuhad illuhad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tomdeakin
Copy link
Contributor

Discussed previously and revisit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants